Alrighty all tests run locally anyway.

I think the reasons the tests are failing is due to the laravel/legacy-factories package requiring a minimum of PHP 7.3 but the tests are running for PHP 7.2 (as that's what Statamic supports).

I would have thought moving it to be a dev dependency would have sorted but doesn't seem like it. 🤔

We're only using a model factory in one file:

https://github.com/statamic/cms/blob/master/tests/Auth/Eloquent/EloquentUserTest.php#L101
https://github.com/statamic/cms/blob/master/tests/Auth/Eloquent/EloquentUserTest.php#L34-L41

We can probably just avoid using a model factory there and then get rid of the dependency.

Having some difficulty w/ fake user generation and non-unique ids

I have opened a PR on your fork which fixes all the failing tests by not using laravel/legacy-factories but instead using Faker on the failing test directly.

PR: https://github.com/edalzell/cms/pull/415

You have merged my PR only "half". You use "WithFaker" Trait in the Test now without using Faker in the "makeUser" method.

Thanks very much for this. 🎉 @mrmonat, too.